Skip to content

[codex] Export WebGL figure initializer#29

Merged
maartenbreddels merged 1 commit intobqplot:masterfrom
maartenbreddels:codex/export-webgl-figure-init
May 8, 2026
Merged

[codex] Export WebGL figure initializer#29
maartenbreddels merged 1 commit intobqplot:masterfrom
maartenbreddels:codex/export-webgl-figure-init

Conversation

@maartenbreddels
Copy link
Copy Markdown
Member

This is a small support PR for downstream bqplot WebGL widget packages.

bqplot 0.13 moved the shared WebGL figure renderer setup into bqplot-gl. Downstream packages such as bqplot-image-gl need to initialize that shared renderer when they render their own WebGL marks, but the initializer currently lives in an internal module and is not exported from the package entry point.

This PR exports the existing WebGL figure initialization utilities from the frontend entry point so downstream widget bundles can call the same setup path as bqplot-gl marks. It also includes GLSL shader sources in the npm package files list. Without those shader files, downstream package builds that install bqplot-gl from npm can fail while resolving raw-loader imports such as ../shaders/scales.glsl.

Validation:

  • jlpm run build:lib
  • jlpm run build:nbextension

The nbextension build completed with existing source-map and bundle-size warnings, but no errors. This PR is opened as a draft because we want to demonstrate the downstream bqplot-image-gl fix against this branch before marking it ready.

Copy link
Copy Markdown
Member

@martinRenou martinRenou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@astrofrog
Copy link
Copy Markdown
Contributor

The bqplot-image-gl CI was working well with this - can you confirm that this is good to go @maartenbreddels ?

@maartenbreddels
Copy link
Copy Markdown
Member Author

Yes, this is good to go

@maartenbreddels maartenbreddels marked this pull request as ready for review May 8, 2026 08:36
@maartenbreddels maartenbreddels merged commit 70528e3 into bqplot:master May 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants